popupMenu: Use SEPARATOR a11y role for SeparatorMenuItems

SeparatorMenuItems are not activatable, so SEPARATOR is a more
appropriate role than MENU_ITEM.

This is also what GTK3 does with GtkSeparatorMenuItem.

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/3773>
(cherry picked from commit eb98f88c7ea2a9a9b0462347d967b5d138d41227)

diff --git a/js/ui/popupMenu.js b/js/ui/popupMenu.js
index 39f6d7802c65abf82429c9119835ac1c5ef97dd5..a068ced5ef94a1402ccc07a4c6355158db5d6696 100644 (file)
--- a/js/ui/popupMenu.js
+++ b/js/ui/popupMenu.js
@@ -308,6 +308,7 @@ class PopupSeparatorMenuItem extends PopupBaseMenuItem {
         this.label = new St.Label({text: text || ''});
         this.add_child(this.label);
         this.label_actor = this.label;
+        this.accessible_role = Atk.Role.SEPARATOR;
 
         this.label.connect('notify::text',
             this._syncVisibility.bind(this));